Skip to content

Comment on stale PRs #36622

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Sep 30, 2020
Merged

Comment on stale PRs #36622

merged 2 commits into from
Sep 30, 2020

Conversation

dsaxton
Copy link
Member

@dsaxton dsaxton commented Sep 25, 2020

Updating the stale PR action to comment in the PR. Should help to automate the process of pinging contributors who go quiet. Also updating to run once daily instead of every six hours.

@WillAyd
Copy link
Member

WillAyd commented Sep 25, 2020

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@WillAyd
Copy link
Member

WillAyd commented Sep 25, 2020

lgtm . so this runs once daily but how often does it comment?

@WillAyd WillAyd added the CI Continuous Integration label Sep 25, 2020
@dsaxton
Copy link
Member Author

dsaxton commented Sep 25, 2020

lgtm . so this runs once daily but how often does it comment?

It should comment only when it marks something as stale, and if something is already marked it'll just skip it (so ideally it'll comment at most once on a PR, unless something is going in and out of a stale state)

@WillAyd WillAyd added this to the 1.2 milestone Sep 30, 2020
@WillAyd
Copy link
Member

WillAyd commented Sep 30, 2020

Cool - let's see how this goes

@WillAyd WillAyd merged commit b89e8c0 into pandas-dev:master Sep 30, 2020
@dsaxton dsaxton deleted the stale-pr-message branch September 30, 2020 17:44
kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants